Skip to content

feat: support HTTP request cancellation - #1541

Merged
jlouvel merged 4 commits into
3.0from
feat/rpc-c3-cancellation
Oct 20, 2026
Merged

jlouvel merged 4 commits into
3.0from
feat/rpc-c3-cancellation

Conversation

@jlouvel

@jlouvel jlouvel commented Oct 18, 2026

Copy link
Copy Markdown
Collaborator

Implements C3 from the RPC extensions design, stacked on C2 (#1540).

  • Adds connector cancellation hooks to Request and delegates them through WrapperRequest.
  • Lets Request.abort/Response.abort cancel an active Netty exchange, including an HTTP/2 stream.
  • Reports stream failures as I/O errors, updates the response status, and notifies an asynchronous response callback of post-header failures.
  • Tests client cancellation, the resulting response callback, read failure, and server write failure.

Validation: mvnw -B -pl org.restlet -Dtest=HttpTrailersTestCase,Http2FullDuplexTestCase,Http2CancellationTestCase test.

@jlouvel
jlouvel added this pull request to stack #1548 October 20, 2026 12:42
@jlouvel
jlouvel force-pushed the feat/rpc-c3-cancellation branch from bf37774 to fd7826f Compare October 20, 2026 13:04
@jlouvel
jlouvel force-pushed the feat/rpc-c3-cancellation branch from fd7826f to 081443f Compare October 20, 2026 13:14
Base automatically changed from feat/rpc-c2-http2-duplex to 3.0 October 20, 2026 13:31
jlouvel and others added 4 commits October 20, 2026 09:31
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Jerome Louvel <[email protected]>
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Jerome Louvel <[email protected]>
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Jerome Louvel <[email protected]>
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Jerome Louvel <[email protected]>
@jlouvel
jlouvel force-pushed the feat/rpc-c3-cancellation branch from 081443f to b841d49 Compare October 20, 2026 13:31
@jlouvel
jlouvel merged commit 22398e0 into 3.0 Oct 20, 2026
1 check passed
@jlouvel
jlouvel deleted the feat/rpc-c3-cancellation branch October 20, 2026 13:31
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants